Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Add Picard's BaitDesigner and ExtractSequences tasks #357

Merged
merged 1 commit into from
Aug 22, 2019
Merged

Add Picard's BaitDesigner and ExtractSequences tasks #357

merged 1 commit into from
Aug 22, 2019

Conversation

mjhipp
Copy link
Contributor

@mjhipp mjhipp commented Aug 7, 2019

Used these tasks in a pipeline and thought they belonged here.

  • Allow Picard to set optional arguments if not defined by user
  • Made output directory required in BaitDesigner task (optional in Picard tool)

@codecov-io
Copy link

codecov-io commented Aug 7, 2019

Codecov Report

Merging #357 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #357   +/-   ##
=======================================
  Coverage   91.84%   91.84%           
=======================================
  Files          31       31           
  Lines        1140     1140           
  Branches       71       71           
=======================================
  Hits         1047     1047           
  Misses         93       93

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fdf8223...4d9b267. Read the comment docs.

@nh13
Copy link
Member

nh13 commented Aug 18, 2019

@mjhipp want to squash your commits and make your commit message a little more concise?

@mjhipp
Copy link
Contributor Author

mjhipp commented Aug 19, 2019

@nh13 sure thing, sorry about that

@mjhipp mjhipp removed their assignment Aug 19, 2019
@nh13 nh13 merged commit 2d84ae1 into fulcrumgenomics:master Aug 22, 2019
@clintval clintval deleted the mh_picard_BaitDesigner branch October 7, 2019 07:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants